Skip to content

Alpine CI#4009

Merged
rocallahan merged 3 commits intorr-debugger:masterfrom
GitMensch:patch-2
Apr 26, 2026
Merged

Alpine CI#4009
rocallahan merged 3 commits intorr-debugger:masterfrom
GitMensch:patch-2

Conversation

@GitMensch
Copy link
Copy Markdown
Contributor

@GitMensch GitMensch commented Aug 24, 2025

  • only to be triggered manually (but to do so the file needs to be available on the main branch)
  • ignores build errors for now, showing all issues instead of only one

@GitMensch GitMensch force-pushed the patch-2 branch 2 times, most recently from 308a4a4 to f80f054 Compare August 24, 2025 14:32
@GitMensch GitMensch marked this pull request as ready for review August 24, 2025 14:37
for now: only to be run manually
[skip ci]
@GitMensch
Copy link
Copy Markdown
Contributor Author

GitMensch commented Aug 25, 2025

Just for reference the CI build from before removing LARGEFILE define: workflow run https://github.com/GitMensch/rr/actions/runs/17189977398/job/48764257227 also providing a build log of only the ninja part and the workflow run after https://github.com/GitMensch/rr/actions/runs/17210712395/job/48821516245 with its ninja build log.

... and an updated one with the open referenced PRs included: https://github.com/GitMensch/rr/actions/runs/17749762907/artifacts/4018389855

@GitMensch
Copy link
Copy Markdown
Contributor Author

@rocallahan Would you merge this workflow definition?
If not: what is missing (other than a squash, but that can be done as part of the merge with a single click as well)?

Its sole purpose is to be run manually from time to time (only the manual workflow-dispatch is enabled) - and any fork working on fixes would also be able to do that once pulled into master (the dispatch only works if the file is in master).

@rocallahan rocallahan marked this pull request as ready for review April 26, 2026 22:08
@rocallahan rocallahan merged commit 7592750 into rr-debugger:master Apr 26, 2026
4 of 5 checks passed
@rocallahan
Copy link
Copy Markdown
Collaborator

Sure seems fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants